menu button: Add a style class

Add a .menu style class to differentiate menu buttons from normal
ones.

scale button: Add a style class

Add a .scale style class to differentiate scale buttons from normal
ones.

link button: Add a style class

Add a .link style class to differentiate link buttons from normal
buttons.

HighContrast: Updates for recent changes

This is bit of a grab-bag, it includes fixes for spin buttons,
entries, buttons.

Adwaita: Adapt to the button changes

The updates here cover the various button variants.

model button: Convert to CSS nodes

Follow the same approach as used for the toggle button family:
Keep the button element name for button-like rendering, and
use a distinct modelbutton name otherwise, and add a subnode
for the indicator with name check or radio.

toggle button: Convert to css nodes

Convert GtkToggleButton and its subclasses to CSS nodes.
Keep the button element name for when we want to render
these button-like (but with .toggle, .check and .radio
style classes for differentiation).

When we want to render them with an indicator, use distinct
element names checkbutton and radiobutton, and add a subnode
for the indicator with name check or radio.

lockbutton: Revisit element name

Instead of a separate element name, keep button, and just
add a .lock style class, to inherit the button theming.

Adwaita: button fixes

togglebutton: Add a style class

This lets us differentiate toggle buttons from plain buttons
with button.toggle, without having to redo all the styling.

button: Use an element name instead of a style class

widget: Queue redraw on gtk_widget_queue_allocate()

Mirror the behavior of gtk_widget_queue_resize() and always queue a
redraw. If we ever want to cause allocates without redraws we can add
gtk_widget_queue_allocate_no_redraw() then.

I had initially assumed gtk_widget_size_allocate() would take care of
queueing redraws, but it does not do that when neither size nor position
change. And that is obviously what's happening after
gtk_widget_queue_allocate().

Fixes buttons sometimes not redrawing (the record button in
widget-factory after locking it, all buttons when switching to the dark
theme).

Assistant: Fix page rmoval

We have to remove the page itself from the intermediate box
first, before removing the box from the notebook. Otherwise,
reffing the page to keep it alive is ineffective: the box
gets destroyed, and that destruction recurses over the page.

This fixes the problem in
https://bugzilla.gnome.org/show_bug.cgi?id=756385

Add a more complex widget in the flipping assistant

This reproduces the problem from bug
https://bugzilla.gnome.org/show_bug.cgi?id=756385

Adwaita: Update double-border tweak for scrolled windows

Since commit b6dbfc791f, GtkViewPort uses an element name.

placessidebar: do not crash if uris is NULL

On Drag'n'Drop actions across system boundaries (VM host to guest), the
happen to be null.

https://bugzilla.gnome.org/show_bug.cgi?id=757298

spinbutton: Don't set .entry

Also fix Adwaita to not use .entry anymore

searchentry: Fix includes

viewport: Add an element name to the css node

This lets us avoid hardcoding the type in CSS.

searchbar: Add an element name to the css node

This lets us avoid hardcoding the type in CSS.

actionbar: Add an element name to the css node

This lets us avoid hardcoding the type in CSS.

statusbar: Add an element name to the css node

This lets us avoid hardcoding the type in CSS.

search entry: Add a style class

There should be little need to have separate styling for search
entries, so a separate name would be a bit overkill.

lock button: Add an element name

entry, spinbutton: Finish css node conversion

This commit creates entry and button subnodes for the buttons
in GtkSpinButton. The nodes are ordered like this for horizontal

  spinbutton
  + entry
    + image.left
    + image.right
    + progress
  + button.down
  + button.up

and like this for vertical ones:

  spinbutton
  + button.down
  + entry
  + button.up

This arrangement requires cooperation from GtkEntry to place
the entry subnodes correctly, and some small changes in the theme.

This commit also fixes progress rendering in vertical spin buttons.

wip: spinbutton

spinbutton: Trivial cleanup

Use g_clear_object in a few places.

widget: Queue resize on parent on show/hide

When gtk_widget_show() or gtk_widget_hide() is called, don't queue a
resize on the widget itself but on the parent.

The widget itself may already be marked as in need of a resize and
the call would be optimized out and never reach the parent.
The parent size will change though because a child widget just changed
its visibility.

Fixes a bunch of issues with menus appearing black, toolbas not hiding
in widget-factory and also various reftests.

container: Don't keep our own variable trying to track need for resize

Insteaad, introduce gtk_widget_needs_allocate() and use that.

widget: Make invisible widgets not propagate resizes

Do not queue a resize on the parent if the widget is not visible.
Invisible widgets do not influence the parents size.

window: Don't queue resizes when inhibiting resizes

Since the frame clock, the inhibit happens on the GDK level, so this
code is no longer necessary.

widget: Make gtk_widget_queue_allocate() not resize

This commit toggles the big switch. We now don't run size_allocate()
from the toplevel up anymore in cases where we don't need to.

Things might be broken in subtle ways as a result of this commit. We'll
have to find them and fix them.

widget: Track if a child needs an allocation

This is not used so far.

widget: Refactor function

Turn it from a loop into tail-calling itself.

widget: Move resize function into gtkwidget.c

There's no sizegroup specific code left in it, but lots of
widget-specifics.

sizegroup: Merge function into only caller

sizegroup: Simplify widget's size group invalidation

Since we do not queue resizes twice anymore, we can just walk the list
of groups without deduplication.

sizegroup: Skip resizes on widgets that have resize queued

Widgets that already have a resize queued don't need to walk the whole
parent chain and queue another resize. It's enough to do it once per
resize.

This also means that sizegroups cannot use the shortcut of just
invalidating the first widget in the group anymore. That widget might
already have a resize queued while others don't.

widget: Warn on calls to queue_resize() during size_allocate()

This happens way too much, so it's disabled unless GTK_DEBUG=geometry is
on.

Also, we can't detect it in the call to queue_resize() yet, only during
size_allocate(), so the warning comes after the signal emission.

widget: Clear pending resizes after size_allocate()

This catches the cases where widgets call gtk_widget_queue_resize()
in their size_allocate() callback.

widget: Warn if size-unqueried widgets get allocated

Widgets should have gtk_widget_get_preferred_width/height() called
on them before gtk_widget_size_allocate() gets called.
Add a check for this.

widget: Add a resize_needed flag

... and API to set and unset it.

It is set when gtk_widget_queue_resize() is called.
It is unset when gtk_widget_get_preferred_width/height() is called.

So far it is not used.

widget: Add a function that does actual resize

It's just shuffling code around.

sizegroup: Merge function into only caller

Both functions were running the same loop. Just running the loop once
seems advantageous.

sizegroup: Don't invalidate over resize containers.

Before this commit, a widget tree like this:
  Window
    AnyContainer (part of SizeGroup1)
      GtkClutterEmbed
        SomeWidget
when calling gtk_widget_queue_resize(SomeWidget), would invalidate
SizeGroup1, when it should have stopped at the GtkClutterEmbed (which is
a RESIZE_IMMEDIATE child).

sizegroup: Refactor function

Get rid of continues in loop. This allows doing more complex things in
future commits.

sizegroup: Refactor function

sizegroup: Remove special case before loop

The loop actually does the right thing already. There's no need to
complicate things.

sizegroup: refactor

The code looks a lot less dangerous if it doesn't look like we're
casting a random widget to a container.

sizegroup: Fold function into only caller

sizegroup: Move container function into its only caller

widget: Use gtk_widget_queue_allocate() when clip changes

There's no need to queue a full resize there.

API: widget: Add gtk_widget_queue_allocate()

This is so widgets can queue a rerun of their allocation logic, but
without triggering resizes everywhere.

For now, it just calls gtk_widget_queue_resize().

container: Use correct allocation for calling size_allocate()

API: widget: Add gtk_widget_get_allocated_size()

See docs for what this is.

sizegroup: Remove GtkQueueResizeFlags

They were only used with geometry widgets.

widget: Remove _gtk_widget_override_size_request()

The function was only used by the geometry widget.

window: Ignore geometry widget

Ignore the geometry widget passed to gtk_window_set_geometry_hints().
Usind the widget itself was a hack that complicates the size request
machinery.

It is also incorrect in that it doesn't respect height-for-width.

Last but not least, it was only used by gnome-terminal and that
application can easily work without it.

container: Don't randomly call gtk_widget_set_allocation()

gtk_widget_size_allocate() does that already.

entry: Always render like with is_cellrenderer

And remove the API to set that variable.
If you want the entry to not fill its whole allocated area,
  gtk_widget_set_valign (entry, GTK_ALIGN_FILL);
will give you the old behavior.

icontheme: Clarify GENERIC_FALLBACK flag handling

- Add docs explaining that it doesn't work everywhere
- g_warn_if_fail() in the APIs where it doesn't work

iconhelper: Don't pass GENERIC_FALLBACK flag

It's not used when looking up with a GIcon.

cssprovider: Have a section when parsing style properties

Signal handlers expect a section to be present, so provide them with
one.

New testcase included.

https://bugzilla.gnome.org/show_bug.cgi?id=757240

Revert "tooltip: Use an element name"

This reverts commit efc8dc63e30d9d53df3cf336673f751fab437d2c.

This change broke tooltip positioning in a way that I don't
fully understand, so take it back for now.

Revert "HighContrast: Update tooltip styling"

This reverts commit 93d80380ee93f2d7f7005675900e4da0fbef982c.

Revert "Adwaita: Update tooltip theming"

This reverts commit d974610741349b3f4e4c06abd87c82d99c6858ba.

Fix key themes for GtkEntry -> entry

This was pointed out in

https://bugzilla.gnome.org/show_bug.cgi?id=757252

shortcuts: Redo the stack switcher theming

Use style classes on the button instead of the stack switcher,
this makes it easier to reuse elsewhere.

shortcuts label: Redo the keycap theming

Use a style class directly on the frame. This is easier to reuse.

Adwaita: Update shortcuts window theming

Use the newly added element name.

shortcuts: Use predefined style classes

Lets use the defines we have.

HighContrast: Update calendar theming

Use the newly introduced element name instead of hardcoding the type.

Adwaita: Update calendar theming

Use the newly introduced element name instead of hardcoding the type.

calendar: Add an element name

This will allow us to drop hardcoded type names in the theme.

wayland: configure clients that resize themselves

Make sure the wayland backend sets a new geometry when the client
resizes itself, otherwise the compositor won't be notified and may
revert to the old size on state changes.

Thanks to Jasper St. Pierre <jstpierre@mecheye.net> who pointed out the
problem in gtk+.

bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=755051

inspector: Cosmetic changes to the hierarchy tab

Put the interfaces below GInterface.

inspector: Add css node style properties

Add a per-cssnode view of style properties.

Drop some no-longer existing headers

3.19.1

Fix reftests

Multiple reftests were damaged by the element name transition.
Repair them. The region reftests are no longer needed, since
we no longer do regions.

HighContrast: Update tooltip styling

Use the newly added element name instead of hardcoding the type.

Adwaita: Update tooltip theming

Use the newly introduced element name instead of hardcoding the type.

tooltip: Use an element name

This will allow us to drop hardcoded type names in the theme.

mir: emulate pointer for all touch events

cssselector: Print name and any selector first

When printing a "compound selector", make sure the name and universal
selectors are printed at the beginning and class, id, etc. selectors are
printed last.

Adwaita: Update GtkAssistant theming

Use the new element name, instead of hardcoding the type.

assistant: Use an element name

This will allow us to drop hardcoded type names in the theme.

Fix make check

Don't try setting GtkShortcutsShortcut::accelerator to a
random value, it only accepts strings that it can parse.

toolitemgroup: Fix the previous change

dispose can be called more than once.

tool item group: Don't leak a button

This causes a hickup in the new css node machinery, causing
the default-value test to fail.

icon theme: Ensure to propagate an error

If the svg pixbuf loader is not available, we end up with criticals
from gtk_css_image_icon_theme_draw because gtk_icon_info_load_symbolic
returns NULL without setting an error.

Avoid this by propagating the load error.

mir: warning clean-up

mir: allow logging touch events

shortcuts: Some property hygiene

Bring property notification for some of the new shortcuts widgets
up to the standards of our testsuite.

gdkpixbuf-drawable: reinstate accidentally removed line

gdk_pixbuf_get_from_window: honor device scale

gdk_pixbuf_get_from_window() paints the given window onto a new cairo
surface. Create that new surface with the same device scale as the
window so that the result is not scaled down on hidpi screens.

https://bugzilla.gnome.org/show_bug.cgi?id=757147

Updated Czech translation

inspector: Allow editing css node properties

This reuses the property editor infrastructure from the
property list pages. Good that css nodes are objects.

inspector: Support editing interned string properties

Sadly, interned string properties cannot be handled generically
at all - GObject insists on inserting a strcpy in any attempt
to set a string property with generic api, destroying the
internedness of the string.

Therefore, we have to special-case GtkCssNode in the property
editor code :-(

Add some api annotations

Nothing is using the /*interned*/ annotations currently,
but if we are doing this, we should be consistent.